-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Research consent form #1164
Research consent form #1164
Conversation
#[generated_doc] | ||
#[instrument(skip(pool))] | ||
pub async fn post_user_consents( | ||
user_id: web::Path<Uuid>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
user: AuthUser
pub deleted_at: Option<DateTime<Utc>>, | ||
} | ||
|
||
pub async fn insert( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upsert
#[generated_doc] | ||
#[instrument(skip(pool))] | ||
pub async fn get_research_consent_by_user_id( | ||
user_id: web::Path<Uuid>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
authuser
@@ -0,0 +1,162 @@ | |||
/* eslint-disable i18next/no-literal-string */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't disable for the whole file
@@ -25,8 +28,10 @@ const Login: React.FC<React.PropsWithChildren<unknown>> = () => { | |||
const [notification, setNotification] = useState<string | null>(null) | |||
const [email, setEmail] = useState("") | |||
const [password, setPassword] = useState("") | |||
const [userId, setUserId] = useState("") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const [userId, setUserId] = useState("") | |
const [userId, setUserId] = useState<string | null>(null) |
if (consent === undefined && optionSelected) { | ||
setOptionSelected(false) | ||
} | ||
const consentQuery = useQuery({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
useToastMutation
router.push(returnTo) | ||
}, [router, uncheckedReturnTo]) | ||
|
||
const getUserConsent = useQuery({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to hooks folder
|
||
return ( | ||
<div> | ||
{/* eslint-disable-next-line i18next/no-literal-string */} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need
> | ||
<div>{t("research-consent-title")}:</div> | ||
<Button | ||
id={"changeReseachConstent"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -99,6 +99,7 @@ | |||
"to-complete-this-chapter": "Suorittaaksesi tämän luvun", | |||
"translate": "Käännä", | |||
"university-of-helsinki": "Helsingin yliopisto", | |||
"user-settings": "Käyttäjäasetukset", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Käyttäjän asetukset
await page.fill(`label:has-text("Password")`, "student-without-research-consent") | ||
await page.locator("id=login-button").click() | ||
await expect(page.locator("text=Organizations")).toBeVisible() | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change the setting
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
932e391
to
8ae30ef
Compare
No description provided.